-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Display verification status as a badge atop the user profile icon (issue241) #243
Conversation
Are you sure? I am able to see the transition from Unknown --> Unverified when starting Robrix, and then after I complete the verification process, I also observe the transtion from Unverified --> Verified. What steps are you taking such that verification states are not updated? Perhaps you can share a quick screen recording video or something. |
I just tried the entire verification procedure again, and it worked as expected. Here is an almost complete log dump:
Grepping for "verification state", you can see the state updates here:
|
If you're not able to reproduce this, then perhaps your matrix account keys weren't set up or preserved correctly? Are you able to self-verify other devices using other non-Robrix clients? |
This func:
robrix/src/verification.rs
Line 20 in 185459f
It seems not work well in here:
robrix/src/sliding_sync.rs
Line 972 in 185459f
I guess it seems the verificating is too earily to be not proper.
You can verify your device & session, and the log is always
Unknown
:robrix/src/verification.rs
Line 22 in 185459f
This pr just make it work , however, I know where I modified is not in your expectation, but it just is a draft.
I have not loaded svgs yet, I just want to prove where there call the func is not working and shall modify it later.
Fixes #241